Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor/#821 메시지 관련 api 명세 수정 #836

Merged
merged 7 commits into from
Nov 10, 2023
Merged

Refactor/#821 메시지 관련 api 명세 수정 #836

merged 7 commits into from
Nov 10, 2023

Conversation

amaran-th
Copy link
Collaborator

#️⃣ 연관된 이슈

close : #821

📝 작업 내용

쪽지 관련 API 명세를 변경

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 11/5
실제 소요 시간 : 11/8

amaran-th added 7 commits November 5, 2023 05:14
refactor: 스크랩 추가/삭제 시 스크랩한 행사 데이터를 반환하도록 수정
946308d 
#823
refactor: 스크랩 삭제 시 스크랩한 행사 데이터를 반환하도록 상태 코드 수정
6ecb9f9 
#823
docs: scrap 관련 api 요청 http 파일 작성
cdf1f0f 
#823
refactor: Message 관련 api 반환 명세 수정
009adec 
- Message가 id와 sender(MemberReferenceResponse)를 추가로 반환하도록 수정
- Room이 Message와 interlocutor(MemberReferenceResponse)를 추가로 반환하도록 수정

#821
refactor: Message 관련 api 반환 명세 수정
18e8866 
- 쪽지방 조회 시 조회하려는 회원의 id를 입력받지 않도록 수정

#821
fix: 스크랩 삭제 시 검증하는 상태코드를 noContent 대신 ok로 수정
9998b29 
#821
fix: N+1 문제 해결
0d180a6 
#821
@amaran-th amaran-th added Backend 백엔드 관련 이슈 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경 High Priority 리뷰 우선순위가 높은 PR labels Nov 8, 2023
@amaran-th amaran-th added this to the 레벨 5 milestone Nov 8, 2023
@amaran-th amaran-th self-assigned this Nov 8, 2023
@amaran-th amaran-th merged commit b9d5cfe into backend-main Nov 10, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@amaran-th amaran-th

Labels
Backend 백엔드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경
Projects
2023-emmsale
Status: Done
Milestone
레벨 5
Development

Successfully merging this pull request may close these issues.
1 participant
@amaran-th